Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hlmem: Add type id to print and allow doing locate on type ids #727

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

Speedphoenix
Copy link
Contributor

This allows doing locates on types containing spaces

With -c, this displays like this
image

And then doing locate #45319 yields things referencing the Function(Void -> Void) type

This allows doing locates on types containing spaces
@ncannasse
Copy link
Member

I think functions should be identified by their implementation instead of their type, this is in discussion with @yuxiaomao

@Speedphoenix
Copy link
Contributor Author

Speedphoenix commented Nov 26, 2024

This is actually less about functions and more about any type that has spaces or that is just very long

For example
image
image

Using heaps' hxsl shaders yields some of these
image
image

There are also some structs that fit on 8 lines or more in fullscreen 1440p like this hxbit object (not the only one)
image

and also some long types without spaces that are fastidious to type or paste without the id
image

Being able to locate functions before their implementation is available is convenient, but more of an added perk than the initial intention

@ncannasse ncannasse merged commit 80036b6 into HaxeFoundation:master Nov 26, 2024
10 checks passed
@Speedphoenix Speedphoenix deleted the hlmem-locateid branch November 27, 2024 11:45
@skial skial mentioned this pull request Dec 9, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants